Skip to content

ENG-8540: avoid dataclasses.asdict in Lost+Found path#6057

Merged
adhami3310 merged 4 commits intomainfrom
masenf/lost-and-found-serialization
Dec 18, 2025
Merged

ENG-8540: avoid dataclasses.asdict in Lost+Found path#6057
adhami3310 merged 4 commits intomainfrom
masenf/lost-and-found-serialization

Conversation

@masenf
Copy link
Copy Markdown
Collaborator

@masenf masenf commented Dec 17, 2025

Use the pickle to serialize LostAndFoundRecord and SocketRecord objects in the token manager.

Use the reflex serializers registry to serialize StateUpdate objects for
Lost+Found usage.
@linear
Copy link
Copy Markdown

linear bot commented Dec 17, 2025

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Dec 17, 2025

CodSpeed Performance Report

Merging #6057 will not alter performance

Comparing masenf/lost-and-found-serialization (3ebdf3e) with main (3beb1a6)

Summary

✅ 8 untouched

@adhami3310 adhami3310 merged commit fcb937c into main Dec 18, 2025
47 checks passed
@adhami3310 adhami3310 deleted the masenf/lost-and-found-serialization branch December 18, 2025 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants